Skip to content

Conversation

@collinfunk
Copy link

This was done in some other python files in the following commit:
4c1e487

Without this change these scripts do not run on Debian or Ubuntu systems. See the referenced patch for more information.

This was done in some other python files in the following
commit:
4c1e487

Without this change these scripts do not run on Debian or
Ubuntu systems. See the refrenced patch for more information.

Signed-off-by: Collin Funk <[email protected]>
@github-actions
Copy link

github-actions bot commented Feb 7, 2024

Thank you for submitting a Pull Request (PR) to the LLVM Project!

This PR will be automatically labeled and the relevant teams will be
notified.

If you wish to, you can add reviewers by using the "Reviewers" section on this page.

If this is not working for you, it is probably because you do not have write
permissions for the repository. In which case you can instead tag reviewers by
name in a comment by using @ followed by their GitHub username.

If you have received no comments on your PR for a week, you can request a review
by "ping"ing the PR by adding a comment “Ping”. The common courtesy "ping" rate
is once a week. Please remember that you are asking for valuable time from other developers.

If you have further questions, they may be answered by the LLVM GitHub User Guide.

You can also ask questions in a comment on this PR, on the LLVM Discord or on the forums.

@llvmbot llvmbot added the clang Clang issues not falling into any other category label Feb 7, 2024
@llvmbot
Copy link
Member

llvmbot commented Feb 7, 2024

@llvm/pr-subscribers-clang

Author: Collin Funk (collinfunk)

Changes

This was done in some other python files in the following commit:
4c1e487

Without this change these scripts do not run on Debian or Ubuntu systems. See the referenced patch for more information.


Full diff: https://github.com/llvm/llvm-project/pull/80977.diff

4 Files Affected:

  • (modified) clang/tools/include-mapping/test.py (+1-1)
  • (modified) clang/tools/scan-view/bin/scan-view (+1-1)
  • (modified) clang/tools/scan-view/share/Reporter.py (+1-1)
  • (modified) clang/tools/scan-view/share/startfile.py (+1-1)
diff --git a/clang/tools/include-mapping/test.py b/clang/tools/include-mapping/test.py
index eef328381f2bb..c0f1f3a79f922 100755
--- a/clang/tools/include-mapping/test.py
+++ b/clang/tools/include-mapping/test.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # ===- test.py -  ---------------------------------------------*- python -*--===#
 #
 # Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
diff --git a/clang/tools/scan-view/bin/scan-view b/clang/tools/scan-view/bin/scan-view
index d01aebb4029a7..f3375804654d2 100755
--- a/clang/tools/scan-view/bin/scan-view
+++ b/clang/tools/scan-view/bin/scan-view
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 
 from __future__ import print_function
 
diff --git a/clang/tools/scan-view/share/Reporter.py b/clang/tools/scan-view/share/Reporter.py
old mode 100644
new mode 100755
index 21874b378687e..aa16a47524c8b
--- a/clang/tools/scan-view/share/Reporter.py
+++ b/clang/tools/scan-view/share/Reporter.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # -*- coding: utf-8 -*-
 
 """Methods for reporting bugs."""
diff --git a/clang/tools/scan-view/share/startfile.py b/clang/tools/scan-view/share/startfile.py
old mode 100644
new mode 100755
index d63e69280e90d..05e88ce642171
--- a/clang/tools/scan-view/share/startfile.py
+++ b/clang/tools/scan-view/share/startfile.py
@@ -1,4 +1,4 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # -*- coding: utf-8 -*-
 
 """Utility for opening a file using the default application in a cross-platform

@collinfunk collinfunk closed this Jun 15, 2025
@collinfunk collinfunk deleted the fix-python3-shebang branch June 15, 2025 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants